Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Add View Cube #27

Closed
wants to merge 1 commit into from
Closed

[WIP] Add View Cube #27

wants to merge 1 commit into from

Conversation

zalo
Copy link
Owner

@zalo zalo commented Oct 4, 2020

This PR begins adding a View Cube at the request of #26.

After this quick and dirty implementation, I'm not totally convinced the additional complexity and state are worth it. The camera movement functionality isn't necessary in a SCAD app, given the drawing occurs entirely from the code-editor and not the graphical editor.

I may downgrade this to a simple orthonormal basis (to provide a quick reference for which direction is which).

This PR begins adding a View Cube at the request of #26.

After this quick and dirty implementation, I'm not totally convinced the additional complexity and state are worth it.  The camera movement functionality isn't necessary in a SCAD app, given the drawing occurs entirely from the code-editor and not the graphical editor.

I may downgrade this to a simple orthonormal basis (to provide a quick reference for which direction is which).
@zalo zalo added the enhancement New feature or request label Oct 4, 2020
@vercel
Copy link

vercel bot commented Oct 4, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/zalo/cascade-studio/3a51hh4oo
✅ Preview: https://cascade-studio-git-feat-view-cube.zalo.vercel.app

@zalo zalo added the wontfix This will not be worked on label Oct 4, 2020
@zalo
Copy link
Owner Author

zalo commented Jan 9, 2021

Going to close this one for now; maybe someday this functionality will come back as keyboard shortcuts.

@zalo zalo closed this Jan 9, 2021
@zalo zalo deleted the feat-view-cube branch January 9, 2021 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant